@melsawy, you should also update the config/routes.rb file, right?

right Caio and also I removed the welcome email and duplicate email

@melsawy I went over the code, but I don't think I had any questions besides the ones @caiosba already asked.

I was trying to manually test it, but I'm having some trouble running the API. I'll poke around, try to fix it, and let you know once I'm able to test it.

@melsawy I was able to run a first test, I tried to register without having an invitation code. But it just failed silently, it didn't notify me of what was happening. Here is a video:

I was also wondering if we shouldn't remove the sign-up option, since we now only allow registration through the invite code, and you can't really just sign-up. What do you think?

I was also wondering if we shouldn't remove the sign-up option, since we now only allow registration through the invite code, and you can't really just sign-up. What do you think?

Already I removed sign-up, you should test using same branch for check-web: CV2-6599-restrict-user-registration-to-invitation-only

I was also wondering if we shouldn't remove the sign-up option, since we now only allow registration through the invite code, and you can't really just sign-up. What do you think?

Already I removed sign-up, you should test using same branch for check-web: CV2-6599-restrict-user-registration-to-invitation-only

I checked, and it looks like I was in the correct branch 🤔 I'll retry this later

@melsawy I was able to run a first test, I tried to register without having an invitation code. But it just failed silently, it didn't notify me of what was happening

@vasconsaurus I raised error here and should handle the message in check-web branch